Skip to content

add additional validation tests#2621

Merged
jmacd merged 14 commits intoopen-telemetry:mainfrom
c1ly:additional-validation-framework-tests
Apr 24, 2026
Merged

add additional validation tests#2621
jmacd merged 14 commits intoopen-telemetry:mainfrom
c1ly:additional-validation-framework-tests

Conversation

@c1ly
Copy link
Copy Markdown
Contributor

@c1ly c1ly commented Apr 9, 2026

Add additional validation tests for testing the transform (kql and opt queries), log sampling, and temporal reaggregation processors

@github-actions github-actions Bot added the rust Pull requests that update Rust code label Apr 9, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.86%. Comparing base (ebf2403) to head (ff94c1d).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2621      +/-   ##
==========================================
- Coverage   87.87%   87.86%   -0.01%     
==========================================
  Files         638      638              
  Lines      244608   244608              
==========================================
- Hits       214942   214935       -7     
- Misses      29142    29149       +7     
  Partials      524      524              
Components Coverage Δ
otap-dataflow 89.90% <ø> (-0.01%) ⬇️
query_abstraction 80.61% <ø> (ø)
query_engine 90.74% <ø> (ø)
otel-arrow-go 51.92% <ø> (ø)
quiver 92.27% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@c1ly c1ly marked this pull request as ready for review April 20, 2026 21:31
@c1ly c1ly requested a review from a team as a code owner April 20, 2026 21:31
@c1ly
Copy link
Copy Markdown
Contributor Author

c1ly commented Apr 20, 2026

@JakeDern @albertlockett I added some validation tests for the processors y'all worked, lmk if I missed anything

Copy link
Copy Markdown
Contributor

@JakeDern JakeDern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @c1ly, it's nice to have a second set of eyes testing these processors!

@github-actions github-actions Bot added the ci-repo Repository maintenance, build, GH workflows, repo cleanup, or other chores label Apr 22, 2026
Copy link
Copy Markdown
Contributor

@drewrelmas drewrelmas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome to see KQL being included in these validation tests!

@jmacd jmacd added this pull request to the merge queue Apr 24, 2026
Merged via the queue into open-telemetry:main with commit c32ead4 Apr 24, 2026
84 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-repo Repository maintenance, build, GH workflows, repo cleanup, or other chores rust Pull requests that update Rust code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants